tidy modernize-deprecated-headers

Merge pull request #673 from tsteven4/tidyupagain

clang tidy readability fixes

tidy readability-qualified-auto

I also ran these through astyle with our astylerc file to match
our standard.  This resulted in significant reformatting in geojson.cc

tidy readability-const-return-type

with a little whitespace cleanup by hand and removal of an
unused deprecated function.

tidy readability-convert-member-functions-to-static

this took a little bit of hand editing for whitespace and
one change that had to be reverted wrt a Functor signature.

trust Qt to strip out characters illegal in xml. (#671)

this was added in Qt 5.11.0.

clang tidy readability-make-member-function-const

Update standards to c++17, Qt >= 5.12, MSVC >= 2017 (#670)

* raise minimum standards to c++17, Qt 5.12, MSVC 2017.

* avoid std::optional::value which didn't work until macOS 10.14.

* avoid std::optional::value which didn't work until macOS 10.14.

* update runner for codeql so we have Qt >= 5.12

* try harder to get codeql working with newer Qt.

* use c++17 fallthrough attribute.

* use c++17 'if constexpr' as suggested by resharper.

* catch mkicondoc up with c++17

cleanup CMakeLists.txt a bit. (#669)

add support for cmake, ninja to Docker images.
get git to ignore various generated files.

fix some codeql warnings. (#663)

* fix some codeql warnings.

* use c++11 z length modifier in format strings with size_t format arguments.

* don't attempt to upload coverage if token is null or not set.

with the github action flow the CODACY_PROJECT_TOKEN is set and null
for foreign pull requests that don't have access to the secrets.

* double protect token

add github actions for CI. (#662)

* add github actions for CI.

* fix shellcheck issues in ci_install_windows.sh

* generalize windows ci build support

cleanup powershell scripts with Invoke-ScriptAnalyzer, including use of
"-Settings CodeFormatting".

create staging directory for windows installer creation.
It is difficult to exclude all the files in the build directory that we don't
want to distribute as the files are a function of the build system.

add windows cmake build.

catch windows installer up with renamed vc redistributable installer. (#667)

catch cmake up (#666)

with recent changes to *.pro files, and
some lost changes to get windows rc files working.

Merge pull request #660 from gpsbabel/win64_only

switch windows release build from 32 bit to 64 bit (only).

Merge pull request #658 from gpsbabel/date_demo

Modernize time handling in compegps, globalsat, hiketech, jtr, magpro…

Merge pull request #657 from gpsbabel/improve-errors

Improve errors by displaying full filename paths on open failures.

turn off selected msvc warnings for c compiler.

switch windows release build from 32 bit to 64 bit (only).

Modernize time handling in compegps, globalsat, hiketech, jtr, magproto, util.

Unfix warning in util.cc since it breaks builds

...on Windows and I don't feel like wrangling in an upgrade.

Fix a dumb warning in garmin_txt.cc.

Use full path name when displaying errors opening files for reads and writes.

This doesn't work for every place where we issue fatal errors, but it gets
the high value formats.

Merge pull request #647 from gpsbabel/robertlipe-patch-1

ozi: don't deinit a write stream if we don't have one.

add ozi track write test cases

backward compatible changes for Qt6 (#656)

* Avoid implicit conversion of int to QChar.

In Qt6 the constructor QChar(int code) is explicit by default.

* update codacy coverage reporter

* move coverage generation to focal.

* use binary codacy coverage reporter

move Qt 5.15 build to 5.15.2. (#652)

update mac, linux docker qtio builds to 5.12.10. (#650)

* update mac Qt 5.12 build to 5.12.10.

* update qtio docker build to qt 5.12.10

fix "Mismatching new/free or malloc/delete" found by CodeQL. (#649)

Create codeql-analysis.yml (#648)

* Create codeql-analysis.yml

* Update codeql-analysis.yml

ozi: don't deinit a write stream if we don't have one.

We can get into close() with a null stream_ TextStream. I think it has to do with ozi_openfile() reopening a file with a new name to match an extension of a certain type so it's renaming a file that hasn't been written. See issue #646

update build to Qt 5.15.1 from 5.15.0 (#642)

eliminate duplicate ignrando test. (#641)

backward compatible changes for Qt6. (#640)

* backward compatible changes for Qt6.

This is mostly QRegExp -> QRegularExpression.
Some QRegExp usage was was replaced by QString::trimmed.
There is also a QNetworkRequest redirect attribute change that is backward
compatible to Qt5.9.

* eliminate usage of unnecessary regular expressions.

* bring serial_unix into Qt6 compliance.

QSet::toList is gone, QSet::values is identical.
qSort is deprecated.

add ability to disable map preview in GUI (#635)

* add CONFIG option to disable map preview feature.

* additional excludes for disabled map preview

Fix potential segmentation faults with exif. (#639)

1. A pointer exif_app_ to an ExifApp on QList<ExifApp> exif_apps was saved.
After this the list was modified.  This can lead to the saved pointer becoming
invalid.
2. The ExifApp structure had a dtor, but no other special functions, i.e.  it
violated the rule of 3 and the rule of 5.  Operations on QList<ExifApp> may
cause an ExifApp on the list to be copied or destroyed.  If an ExifApp is
destroyed, then the ExifApp dtor would close the files, even though there could
be a copy of the ExifApp expecting the files still to be open.

This scenerio occured with Qt6, causing segmentation faults in exif.test.

fix hiketech datetime bug, prepare for toTime_t override (#638)

* correct hiketech written time values.

the mystery previously noted in the code was due to the use of the %I format
specifier which prints out the hour using a 12 hour clock.

note that it is easy to see the times in the reference files were wrong.  the
times in the first reference file, expertgpx.gpx, are in UTC.  This is always
true for gpx, and is indicated as well by the Z.  the times in the hiketech file
appear in a utc element, so they must be in UTC as well.

this eliminates the usage of QDateTime::toTime_t which is obsolete.

* elimnate remaining uses of obsolete QDateTime::toTime_t

This sounds better than it is, we added uses of gpsbabel::DateTime::toTime_t.
But we control that and can add our own toTime_t.

remove unintentional collision with errno. (#636)

this caused compiler failures with gnu++1z.

more gperf removal (#634)

* more gperf removal.

This also makes the xcsv_token table private.

* restore functionality of style file options

and avoid unnecessary enum conversions.

* expicitly pass default value for xcsv_tokens value lookup.

merge selected upstream changes for upload.sh (#633)

this restores uploads for PRs, but to transfersh.com instead of transfer.sh.

add atrifactory deploy script to upload Qt images.

Allow client retry on a redirect of an upgrade check.
Not in this CL, server side change to NOT force an http->https upgrade
on this one URL.

Merge branch 'master' of https://github.com/gpsbabel/gpsbabel

Remove dependency on gperf. (#631)

* Remove dependency on gperf.

There may be more "computer sciency" ways to do this, but let's start here.
Smallest cut possible for removal. Throwing this into CI to plan for actual
removal.

It seems like xcsv_tokens should be more const-able (constexpr?). If necessary,
we can defer this initialization from a global created at startup to being
only inflated when needed. Let's see.

* Bludgeon in temporary better handling for LAT_10E and friends.

* Prepare to die!

* Drop xcsv_tokens.cc and xcsv_tokens.gperf.

* Drop xcsv_tokens.in.

* Remove Steven's shiny new code to run gperf.

Collision with trunk. Oops.

* Pick up xcsv.cc, xcsv.h which were dropped.

Migrate some RegExs to QRegularExpression. (#632)

QRegExp will be/is deprecated.

add fedora CI build, drop xenial (#630)

* add fedora CI build

* drop travis xenial build.

compile in gpsbabel translations and map on non mac unix distribution… (#622)

* compile in gpsbabel translations and map on non mac unix distributions by default.

This eliminates the need to install gpsbabelfe_*.qm, gpsbabel_*.qm and
gmapbase.html on the system.
Note that the Qt supplied translations must still be installed on the system.
The Qt translations may be in a package like qttranslations5-l10n or
qt5-qttranslations.

* simplify optional resources using std methods.

* discuss dependencies in INSTALL.

add some explicit docbook ids (#629)

this makes the generated html more reproducible as random ids
aren't generated.

remove autotools (#626)

add macos qt 5.15.0 build with artifactory image. (#624)

move linux CI from configure to qmake. (#613)

- Add recipe for internal_styles.cc, xcsv_tokens.gperf, gui, unix-gui, toolinfo to GPSBabel.pro.
- Add support for text substitution in setup.iss and gbversion.h to GPSBabel.pro.
- fix qmake warnings from "qmake -Wall".
- split recipes for documents into scripts.
- shellcheck fixes for fixdoc, mkcapabilities.
- parallel build for coverage.
- add INSTALL file with build instructions.
- sort document targets into ones for general consumption (gpsbabel.html, gpsbabel.pdf) and one to support www.gpsbabel.org (now gpsbabel.org instead of index.html).

fix i386 regression error with engima. (#620)

this was motivated by the test failure seen at
https://buildd.debian.org/status/fetch.php?pkg=gpsbabel&arch=i386&ver=1.7.0%2Bds-4&stamp=1596794554&raw=0

use Qt endian detetion. (#619)

This frees us from AC_C_BIGENDIAN which conditionally defined WORDS_BIGENDIAN.
The defintion of WORDS_BIGENDIAN was and is not handled by GPSBabel.pro.

fix endianness/packing issues with energympro, globalsat_sport. (#618)

this was motivated by test failures seen at
https://buildd.debian.org/status/fetch.php?pkg=gpsbabel&arch=s390x&ver=1.7.0%2Bds-3&stamp=1594393020&raw=0

cleanup README.md (#614)

* update travis url to reflect move from .org to .com

* remove duplicate codacy badge and fix markup error.

* add appveyor status badge.

fix a few gcc10 warnings.

fix lupdate warning. (#617)

With Qt 5.12.9 the lupdate command in package_app could print a warning:
/home/strabert/work/pkgconfig/gui/preferences.cc:28: Discarding unconsumed meta data

Fix NMEA parsing creating Null Island waypoints (#607)

* Fix NMEA parsing creating Null Island waypoints

Fix parsing of NMEA GPGGA sentences: ones with 0 "fix quality" (i.e. "invalid") are allowed in serial (i.e. live GPS) mode because (according to the comment) some GPS devices will report previously-read data in the absence of a current good fix. Adjust this allowance to require an actual coordinate value; at least one popular USB GPS device will issue GPGGA such as "$GPGGA,010222.00,,,,,0,00,99.99,,,,,,*65" if it loses a good fix, and the empty lat/lng coordinates (",,,,,") are parsed as 0N 0W (a.k.a. "Null Island"). QString::toDouble() won't report a problem with an empty input so we simply check for 0 lat/lng.

* Use "ok" QString::toDouble() argument instead

... of checking for checking for both lat & lng exactly zero, as `ok` does indeed get set to `false` on empty inputs, in recent Qt versions.

* Revert "Use "ok" QString::toDouble() argument instead"

This reverts commit d226d01c862606615aa599f5d3b649cea12f0700.

fix error detection for osx script, allow macOS sed for mkstyle. (#615)

support doc creation in qmake. (#609)

* support doc creation in qmake.

pass parameters to tools/mkcapabilites instead of
using text replacement in configure.

text replacement in xmldoc/makedoc was unnecessary, directory
creation is handled in the recipee.

there is some foolishness in GPSBabel.pro to allow overriding of
WEB and DOCVERSION on the command line of make.  If those values
are known when running qmake it could be slightly simpler.

* remove dependency on makedoc.in.

* improve fop comments.

* further update fop comments

support parallel vtesto from qmake. (#612)

* support parallel vtesto from qmake.

also support testo, vtesto with out of source builds.

* force check to run.

support third party library configuration in qmake (#611)

* add qmake options to specify libraries.

* fix no libusb along lines of #610

* correct inclusion of shapelib headers.

* more closely match original includes

instead of trying to use HAVE_*, ENABLED_* *_INHIBITED and
-I options in a consistent manner.

We can revisit this after the demise of configure.

* add qmake message when libusb is disabled.

Update gpsusbstub.cc (#610)

Build fails when configured `--without-libusb`:
```
jeeps/gpsusbstub.cc:39:3: error: ‘Fatal’ was not declared in this scope
   Fatal() << no_usb;
   ^~~~~
jeeps/gpsusbstub.cc:39:3: note: suggested alternative: ‘fatal’
   Fatal() << no_usb;
   ^~~~~
   fatal
```

fix spelling errors flagged by https://lintian.debian.org/sources/gpsbabel/1.6.0+ds-11.html (#604)

also remove gpsbabel executable that should never have been checked in.

Replace xxx with nnn for clarity

Release candidate (#602)

* prep for release 1.7.0.

* update reference files for release.

* use windows resource compiler scripts for CLI and GUI.

These rc scripts are used to set the icon the OS sees and to supply
version information that is shown by windows explorer.

* correct file permissions.

* update rc copyright

Prevent crash building PDF on MacOS when there's not a missing
SOMETHING to go with this ID tag. We don't seem to need it. Gone.

Write garmin specific ilinks as gpxx:rpt elements (#598)

* Write garmin specific ilinks as gpxx:rpt elements when garminextensions option is specified

* incorporate code review comments

* comment corrected  to reflect element being closed

yum -> apt in Ubuntu instructions (#599)

update macos, linux builds to use Qt 5.12.9 (#596)

* switch macos to Qt 5.12.9

* rebuild docker images

Adjust gdb reader duration handling (#595)

* experiment with filtering gdb waypoints.

* formalize new gdb option to avoid creation of waypoints
from non-user points.

Also, don't fill in the description when processing durations.  This
interacted undesirably with the roadbook option.

* fix reference files.

* simplify bool option processing in gdb, new ref files.

* fix reference files again.

* contrast the gdb dropwpt option with the via option

clean up some cppcheck warnings in gdb. (#594)

Extract expected travel durations from Garmin BaseCamp GDB output (#585)

* extract track segment expected travel durations from Garmin BaseCamp autorouted GDB exports and report as <rtept/src> elements in gpx output

* revised to use Garmin Format-Specific Data object to store durations

* generate at frozen test time

* include code review suggestions

* store Garmin BaseCamp autoroute waypoint travel durations in the <desc> elements of <rtept> waypoints; modify gpx ouptut to exclude autoroute generated rtept waypoints from initial waypoint list

* remove waypoint class lookup and replace tabs with spaces

* run astyle on gdb.cc

* turn off detailed GDB debugging

Co-authored-by: tsteven4 <13596209+tsteven4@users.noreply.github.com>

fix Uncaught TypeError: map.checkResize is not a function (#593)

* fix Uncaught TypeError: map.checkResize is not a function

with GUI map preview.

* remove extra newline from map preview javascript.

* tighten up js generation.

* tighten up js some more.

* tweak js generation.

fix segmentation fault in GUI map preview. (#592)

refactor logging (#582)

* refactor logging to

allow the noreturn attribute on fatal() to be used by analysis
tools while still generating messages with an output stream.

* Add our own message handler.

This allows us to use stderr on windows.  The default message handler
routes QDebug messages to the debugger on windows.

* flush stdout before message logger writes to stderr.

* flush stdout before writing to standard error for fatal().

or else messages can appear in the wrong order on Windows.

Remove Mapsource .mps format (#588)

* Fix numerous instances of shadowing confusion.

Inspired by a cppcon talk and our recent conversations about warnings, I
took a closer look at shadowing variables in our code and I wasn't proud
of my findings.

We had a lot of code that was writing to local variables or function
arguments when it thought it was operating on globals or (soon)
method-member variables. I made a rather half-hearted pass at changing
this by splitting up huge functions, renaming arguments, renaming globals
to reflect their future status as method variables (foo_), and generally
cleaning things up. The amound of effort I spent on each format was
approximately equal to its expected lifespan - I'm not spending an hour
on cleanups on formats I should instead be deleting.

In a future submit, I'll try to find some combination of -W flags on
our targets that allows us to not regress easily.

* Fix merge conflict I just created in gdb.cc

* Delete workspace.xml

* Delete the same link in garmin_icons.xml that I deleted twice yesterday.
Fix more esoteric warnings.

* Reapply MORE of yesterday's commits.
Seriously, I know this has been committed and is in HEAD...

* Attempt to fix list detachment within route.cc

* Remove Mapsource

The Mapsource format has been deprecated by Garmin for something
like 15 years. Later versions of Mapsource used GDB and Basecamp,
Garmin's first replacement for the Mapsource lineage, was used
exclusively in later versions.

I've reviewed list traffic and floated a discussion on the list.
The code style was never really very flexible and it it bears
a high maintenance cost that we can no longer afford. This
format is dead.

* Remove more pieces of Mapsource.

* Change Netstumbler test to not use Mapsource for staging.
More cleanups.

* Fix netstumbler so I can delete Mapsource. Ugh.

* Update netstumbler with IWYU.

use http for upgrade (#587)

* test "TLS initialization failed" issue on windows.

* https ok with desktop services.

fix Qt 5.15.0 Wdeprecated-declarations warnings for (#584)

QXmlDefaultHandler
QXmlInputSource
QXmlSimpleReader

as suggested use QXmlStreamReader instead.

convert osm to format class (#579)

* convert osm to Format class.

* use static function cb w/o lambdas.

clean up mkstyle.sh (#578)

as suggested by shellcheck used $() instead of backticks,
and double quote to prevent globbing and word splitting.
mkstyle.sh now works if the directory path contains a space.

drop the failed attempt to exclude custom.style. This resolves #577
drop the exclusion of README.style, which hasn't existed for a long time.

use basename to drop the suffix instead of piping to sed.

fix Qt 5.15.0 Wdeprecated-declarations warnings for (#583)

QTime::start()
QTime::elapsed()

make garmin_tables const (#576)

and clean up mkicondoc cppcheck errors.

Merge pull request #574 from tsteven4/guict

clang-tidy fixes for GUI

clang-tidy readability-isolate-declaration on gui.

clang tidy modernize-use-auto on gui.

clang tidy modernize-redundant-void-arg on gui.

clang-tidy readability-redundant-member-init on gui.

Fix numerous instances of shadowing confusion. (#570)

Fix numerous instances of shadowing confusion.

Inspired by a cppcon talk and our recent conversations about warnings, I
took a closer look at shadowing variables in our code and I wasn't proud
of my findings.

We had a lot of code that was writing to local variables or function
arguments when it thought it was operating on globals or (soon)
method-member variables. I made a rather half-hearted pass at changing
this by splitting up huge functions, renaming arguments, renaming globals
to reflect their future status as method variables (foo_), and generally
cleaning things up. The amound of effort I spent on each format was
approximately equal to its expected lifespan - I'm not spending an hour
on cleanups on formats I should instead be deleting.

In a future submit, I'll try to find some combination of -W flags on
our targets that allows us to not regress easily.

Fail configuration if libusb-1.0 is not found (#564)

* have configure fail if libusb is enabled and not found.

* correct indentation in configure.

Use strncmp to compare array to string (#545)

fix -Wsizeof-pointer-memaccess in garmin_gpi reader. (#569)

again, this really wasn't a problem.  The destination and the
source had the same size.  But we can do better.

Merge pull request #567 from gpsbabel/bye_psitrex

Remove PSITrex, which has been unused for many years.

Merge branch 'bye_psitrex' of https://github.com/gpsbabel/gpsbabel into bye_psitrex

Don't refer to PSITtext in reference files.

More remnants of PSITrex.

Garmin_icons in doc can't refer to psitrex any longer.

Cleanup of PSITrex removal

Cleanup of PSITrex removal

Cleanup of PSITrex removal

Actually prepare PSITrex deletion for cleanup

fix -Wstringop-truncation warning in osm. (#568)